Skip to content

Fix tests#669

Merged
regisss merged 1 commit into
mainfrom
sasarkar/test_fix
Jan 30, 2024
Merged

Fix tests#669
regisss merged 1 commit into
mainfrom
sasarkar/test_fix

Conversation

@ssarkar2
Copy link
Copy Markdown
Contributor

What does this PR do?

Certain tests like these here calls greedy_search and beam_search directly without generate

flags like reduce_recompile and bucket_size are set to default values here, hence they were accessed without defaults in greedy_search and beam_search. However of the *_search functions are called directly (like in the unit tests), it can fail.

Therefore setting defaults in this PR

Fixes # (issue)

Before submitting

  • This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
  • Did you make sure to update the documentation with your changes?
  • Did you write any new necessary tests?

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Copy Markdown
Collaborator

@regisss regisss left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@regisss regisss merged commit ada8b22 into main Jan 30, 2024
@regisss regisss deleted the sasarkar/test_fix branch January 30, 2024 21:57
bhargaveede pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Feb 9, 2024
regisss pushed a commit that referenced this pull request Feb 12, 2024
bhargaveede pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Feb 14, 2024
vivekgoe pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Feb 15, 2024
Co-authored-by: Sayantan Sarkar <supersarkar@gmail.com>
jychen21 pushed a commit to jychen21/optimum-habana that referenced this pull request Feb 27, 2024
HolyFalafel pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Mar 11, 2024
gplutop7 pushed a commit to HabanaAI/optimum-habana-fork that referenced this pull request Oct 15, 2025
…ngface#669)

Co-authored-by: Iman Gohari <s.m.iman.gohari@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants